Skip to content

Feat/issue #22 - 알림 우선순위#23

Merged
Richter3766 merged 10 commits intomainfrom
Feat/issue-#22
Feb 27, 2025
Merged

Feat/issue #22 - 알림 우선순위#23
Richter3766 merged 10 commits intomainfrom
Feat/issue-#22

Conversation

@Richter3766
Copy link
Owner

@Richter3766 Richter3766 commented Feb 27, 2025

PR

작업 내용

  • 앱, 제목, 알림 리스트에 우선순위 적용
  • 우선순위 설정, 취소를 위한 모달창 추가

참고 사항

우선순위 기능 하나로 충분히 PR이 커져서
새로고침 및 우선순위 순서 변경은 추후 이슈로 해결 예정입니다.
UI 관련 코드는 커버리지에서 제외하였습니다.
이는 모바일에서 직접 테스트하는 게 더 유효하다고 보았기 때문입니다.

✏ Git Close

#22

@Richter3766
feat: 데이터 모델 우선순위 내용 추가
445c18f
@Richter3766
feat: 데이터 repository 분리 및 UseCase 추가
b2c2d25 
App, title, 알림에 대해 SRP 적용하여, 각 클래스를 담당하도록 분리.
우선순위를 적절히 바꿀 수 있는 useCase 추가
@Richter3766
feat: 변경된 interface에 맞게 hilt 재적용
22dbfc6
@Richter3766
fix: update 쿼리 리턴값 수정
08176a2 
update 쿼리는 업데이트 성공한 행 수를 Int로 반환함.
따라서 Long -> Int로 수정
@Richter3766
feat: add 더보기 클릭 시 모달창
8f915a1 
앱, 제목 리스트에서 더보기 클릭 시
하단을 채우는 BottomSheet 적용.
해당 칸을 채우는 BoxView 추가.
@Richter3766
feat: 앱 리스트 우선순위 기능 적용
df41958
@Richter3766
feat: 알림 제목 리스트 우선순위 기능 적용.
1d7baf2 
올바른 적용을 위해, 알림 제목 dto에 id 필드를 추가함.
이 값으로 우선순위 추가, 삭제를 관리.
@Richter3766
Test: 우선순위 추가 및 인터페이스 분리에 따른 테스트 수정
d512b62 
json도 수정함.
@Richter3766
feat: 컬러풀 앱 아이콘 로직
96c9c7a 
smallIcon에 색을 입힘으로써 앱 아이콘 표현.
이후 패키지 앱 아이콘을 가져오는 방식과 비교해볼 예정
일종의 테스트 커밋.
@Richter3766
build: kover에서 presentation 제외
3699e5f 
UI 로직은 실제 모바일에서 테스트하므로,
굳이 커버리지에 넣지 않음.
@Richter3766 Richter3766 added the enhancement New feature or request label Feb 27, 2025
@Richter3766 Richter3766 self-assigned this Feb 27, 2025
@Richter3766 Richter3766 linked an issue Feb 27, 2025 that may be closed by this pull request
feat: 알림 우선순위 #22
Closed
2 tasks
@github-actions
Copy link

github-actions bot commented Feb 27, 2025

Code Coverage

File Coverage [59.51%] ✅
app/src/main/java/com/example/notimanager/data/dto/NotificationAppDto.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/dto/NotificationTitleDto.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/model/AppIconModel.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/model/NotificationIconModel.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/repository/NotificationAppRepository.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/repository/NotificationDomainRepository.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/repository/NotificationTitleRepository.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/service/NotiListenerService.kt 0.00% ❌
app/src/main/java/com/example/notimanager/data/utils/AppIconGetter.kt 0.00% ❌
app/src/main/java/com/example/notimanager/domain/model/NotificationApp.kt 100.00% ✅
app/src/main/java/com/example/notimanager/domain/model/NotificationTitle.kt 100.00% ✅
app/src/main/java/com/example/notimanager/domain/usecase/NotificationAppUseCase.kt 50.00% ✅
app/src/main/java/com/example/notimanager/domain/usecase/NotificationTitleUseCase.kt 50.00% ✅
app/src/main/java/com/example/notimanager/domain/usecase/NotificationUseCase.kt 100.00% ✅
Total Project Coverage 54.95% ✅

@Richter3766 Richter3766 merged commit f38c3cb into main Feb 27, 2025
1 check passed
@Richter3766 Richter3766 deleted the Feat/issue-#22 branch February 27, 2025 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Richter3766 Richter3766

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat: 알림 우선순위

1 participant

@Richter3766